feat: hideStep på nve-step som gjør at man kan skjule et steg #523
Conversation
Citaborg
requested review from
NVEJoel,
amish1188,
gruble and
tomapedersen
as code owners
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-523.westeurope.5.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-523.westeurope.5.azurestaticapps.net |
gruble
left a comment
gruble
left a comment
There was a problem hiding this comment.
Kode og dokumentasjon så bra ut.
Kan du bare sjekke hvorfor Vue-eksemplet på hideStep ikke funker?
|
|
||
| </SandboxPreview> | ||
|
|
||
| ### Skjul bestemte steg i Vue |
There was a problem hiding this comment.
Selv om jeg klikker på "Skul steg 2", vises steget fortsatt i PR-bygget.
Er det brukerfeil, feil i koden eller noe annet?
There was a problem hiding this comment.
For meg vises ikke steg 2 i den automatiske genererte preview'en, altså den har "hideStep":true og er da skjult. Kan du se igjen om det fortsatt gjelder for deg?
There was a problem hiding this comment.
Jeg ser også fortsatt steg 2 selvom jeg har klikket på "Skjul steget" i CodeSandboxen. Det fungerer veldig fint i eksempelet under Skjule bestemte steg, men ikke under Vue eksempler > Skjul bestemte steg i Vue. Fungerer det også der for deg @lisamarimyreneNVE?
There was a problem hiding this comment.
Jeg fikk samme problem når det kjørte under vue. Har forsøkt diverse varianter uten å få det til å fungere.
I prosjektet ble det "løst" ved å bytte/endre hele komponenten når man endret innholdet.
Denne PRen legger til en property
hideStepinve-stepkomponenten som gjør at man kan skjule et steg.Endringer:
nve-stepper.component.ts: tar hensyn til hideStep i beregning av hvilket steg som er neste, forrige, osv. Videresender hideStep fra steps-kolleksjonen tilnve-stepnve-step.component.ts: Her er det lagt til en propertyhideStep. I render-metoden (horisontal og vertikal) er sjekkes det somhideSteper satt og hvis den er det så rendres ikke elementet. OppdatertehideDescrtiptionstilhideDescriptionfornve-step(den heterhideDescriptionspånve-stepper). ognve-step: Endret navn på css-klassenempty-descriptiontilstep-description-empty.